Skip to content

[stable30] fix(Wopi): fall back to super share if share token is not available#4738

Merged
elzody merged 1 commit into
stable30from
backport/4712/stable30
May 7, 2025
Merged

[stable30] fix(Wopi): fall back to super share if share token is not available#4738
elzody merged 1 commit into
stable30from
backport/4712/stable30

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot Bot commented May 7, 2025

Backport of #4712

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot Bot requested review from blizzz, elzody and juliusknorr May 7, 2025 08:25
@backportbot backportbot Bot added bug Something isn't working 3. to review Ready to be reviewed labels May 7, 2025
@blizzz blizzz force-pushed the backport/4712/stable30 branch from 7f5ee71 to 8dfaaff Compare May 7, 2025 10:50
On internal shares the controller is called without the share token. But
necessary information, like share attributes, might be necessary to know
and are available from the super share of the SharedStorage in that case.

For this approach was used elsewhere, too, some repetitive code was
consolidated in the Helper class.

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzz blizzz force-pushed the backport/4712/stable30 branch from 8dfaaff to 650fe7c Compare May 7, 2025 10:55
@blizzz blizzz marked this pull request as ready for review May 7, 2025 10:59
@elzody elzody merged commit 6d6aec3 into stable30 May 7, 2025
67 checks passed
@elzody elzody deleted the backport/4712/stable30 branch May 7, 2025 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Ready to be reviewed bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants